Feat/enhanced metrics#644
Conversation
and test snapshots for resolvers
Co-authored-by: Mohammed Izzy <mohammed@serverlessguru.com>
|
Request this branch be merged to master for next release |
2ac9761 to
95e9926
Compare
|
Commit message amended to follow semantic release |
|
@DoyleBrendan, please fix merge conflicts. |
|
Fresh PR - #711 Thanks for this, @DoyleBrendan — enhanced metrics is a genuinely useful addition and your structure made it easy to pick up. I've opened #711 to carry it forward, since this branch had drifted a long way from master (100+ commits, plus a jest 27→30 bump) and an in-place update wasn't practical. Your work is credited as co-author on the commit. OperationLevelMetricsConfig had a leading space in ' DISABLED' (type + validation), so the correct AWS value DISABLED was actually being rejected — fixed. I also added unit + e2e (CloudFormation synthesis) coverage for the API- and resolver-level behavior. Closing this in favor of #711 — please take a look there, and thank you for kicking it off. |
Looking to add basic support for Enhanced Metrics